Skip to content

revert(kyverno): drop temporary background-controller v=4 verbosity#2400

Merged
devantler merged 1 commit into
mainfrom
claude/kyverno-revert-v4-verbosity
Jul 3, 2026
Merged

revert(kyverno): drop temporary background-controller v=4 verbosity#2400
devantler merged 1 commit into
mainfrom
claude/kyverno-revert-v4-verbosity

Conversation

@devantler

@devantler devantler commented Jul 3, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

🤖 Generated by the Daily AI Assistant

Why

While diagnosing why the storage automation never ran on autoscaled nodes (#2392), we temporarily turned up the policy engine's diagnostic logging. The root cause is now fixed and confirmed working in prod, so the extra verbosity is just noise and log volume.

What

Returns the policy engine's logging to its normal level. Nothing else changes.

@devantler @claude
revert(kyverno): drop temporary background-controller v=4 verbosity
b7ae9de 
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@github-project-automation github-project-automation Bot moved this to 🫴 Ready in 🌊 Project Board Jul 3, 2026
@coderabbitai

coderabbitai Bot commented Jul 3, 2026 •
edited
Loading

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

Removes a temporary extraArgs verbosity override (v: 4) from the backgroundController section in the Kyverno HelmRelease values file, which had been added for diagnosing a prior issue.

Changes

Kyverno HelmRelease Cleanup

Layer / File(s) Summary
Remove backgroundController extraArgs override
k8s/bases/infrastructure/controllers/kyverno/helm-release.yaml		 Removes the temporary extraArgs block that set logging verbosity to v: 4 on backgroundController, restoring direct flow from replicas to podDisruptionBudget settings.

Estimated code review effort: 1 (Trivial) | ~2 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The title clearly describes reverting the temporary Kyverno background-controller verbosity change.
Description check ✅ Passed The description is directly related to removing the temporary logging verbosity override and matches the change.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch claude/kyverno-revert-v4-verbosity

Comment @coderabbitai help to get the list of available commands.

@devantler devantler mentioned this pull request Jul 3, 2026
Closed
@devantler

devantler commented Jul 3, 2026

Copy link
Copy Markdown
Contributor Author

🤖 Generated by the Daily AI Assistant

@coderabbitai review

@coderabbitai

coderabbitai Bot commented Jul 3, 2026 •
edited
Loading

Copy link
Copy Markdown

@devantler I'll review the changes now.

✅ Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@devantler devantler marked this pull request as ready for review July 3, 2026 03:47
@devantler devantler added this pull request to the merge queue Jul 3, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jul 3, 2026
@devantler devantler mentioned this pull request Jul 3, 2026
Merged
@devantler devantler added this pull request to the merge queue Jul 3, 2026
@devantler

devantler commented Jul 3, 2026

Copy link
Copy Markdown
Contributor Author

🤖 Generated by the Daily AI Assistant

Heads-up before more re-queues: the 03:48Z eviction (and the failed Heal-Prod) was not a flake — #2399's second matchOIDCIdentity entry broke cosign verification on all three tenant OCIRepositories (unsupported: multiple identities are not supported at this time), so the apps Kustomization can't reconcile and every merge_group deploy fails until that is fixed. The root-cause fix is #2404 (single regex-alternated identity, green + CLEAN, awaiting promotion). The 05:26Z re-queue of this PR (and #2403 behind it) will evict again for the same reason — land #2404 first, then re-queue these.

@devantler devantler mentioned this pull request Jul 3, 2026
Closed
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jul 3, 2026
@devantler devantler added this pull request to the merge queue Jul 3, 2026
Merged via the queue into main with commit 4ecb231 Jul 3, 2026
15 checks passed
@devantler devantler deleted the claude/kyverno-revert-v4-verbosity branch July 3, 2026 06:46
@github-project-automation github-project-automation Bot moved this from 🫴 Ready to ✅ Done in 🌊 Project Board Jul 3, 2026
@botantler-1

botantler-1 Bot commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 1.95.2 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@botantler-1 botantler-1 Bot added the released label Jul 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@coderabbitai coderabbitai[bot] coderabbitai[bot] approved these changes

Assignees

No one assigned

Labels

released

Projects

🌊 Project Board
Status: ✅ Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@devantler